Check out this pull request on 

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

Note:

I am interested in pushing some of the content of these notebooks on PlasmaPy.

I believe the core functions would need to be extracted and implemented in separate .py files.

Hopefully, some of this is of interest to you:

• For example, the notebook "critical_density.ipynb" aims to implement issue Implement the plasma critical density in the formulary #26.
• The "assignment_answers.ipynb" notebook uses a recent Inertial Confinement Fusion experiment at NIF,(https://doi.org/10.1038/s41566-021-00890-z) which made a lot of noise in the Fusion community, to illustrate several of the functions implemented in the other files. This may be a good example for issue Create educational Jupyter notebooks #61.

I have a question about the difference between Pull Request and Drafts:

Can you see Drafts and comment on them?

Is it annoying for you if the Pull Request contains code that is quite rough?

@cmoissar drafts just have a note so we know they aren't ready to review. Absolutely no problem opening a PR even when rough - it allows us to easily see what you're working on and give suggestions.

I think adding some functions for LP HEDP and an example notebook that features them would be great! I would do at as 1 PR per function and then 1 PR for the notebook. If you're still interested, you should open those PRs on the main PlasmaPy package.

Hey @cmoissar, do you mind if I take a stab at implementing the plasma critical density formulary as my first commit?

Hey @cmoissar, do you mind if I take a stab at implementing the plasma critical density formulary as my first commit?

Of course not, have at it!

@cmoissar drafts just have a note so we know they aren't ready to review. Absolutely no problem opening a PR even when rough - it allows us to easily see what you're working on and give suggestions.

I think adding some functions for LP HEDP and an example notebook that features them would be great! I would do at as 1 PR per function and then 1 PR for the notebook. If you're still interested, you should open those PRs on the main PlasmaPy package.

Thanks for your reply.
I'll (slowly) work on that after my summer break!